Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

feat(dashboard): integrations update and create flow #7281

Merged
merged 124 commits into from
Dec 20, 2024
Merged

Conversation

scopsy
Copy link
Contributor

@scopsy scopsy commented Dec 11, 2024

What changed? Why was the change needed?

  • Adds update sidebar
  • Create integration sidebar
  • Handling primary channel switch

Screenshots

CleanShot 2024-12-16 at 19 30 02@2x
CleanShot 2024-12-16 at 19 29 55@2x

Expand for optional sections

Related enterprise PR

Special notes for your reviewer

</TabsContent>
<TabsContent value="data-warehouse" variant="regular">
<div className="text-muted-foreground flex h-64 items-center justify-center">Coming soon</div>
</TabsContent>
</Tabs>
<Outlet />
Copy link
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

to build a nice animation on open and close, you can do a similar thing to what we have in: configure-step-template.tsx plus using the AnimatedOutlet instead of the regular one

Copy link
Contributor Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Will skip the added animation for now, but I like the idea!

isOpened: boolean;
};

export function CreateIntegrationSidebar({ isOpened }: CreateIntegrationSidebarProps) {
Copy link
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

something weird is happening when clicking on connect integration

Screen.Recording.2024-12-20.at.09.45.53.mov

Copy link
Contributor Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Not able to reproduce, might be some local thing going on. I will deploy this to staging and will see if it's still noticable

mode: 'create',
});

async function handleCreateIntegration(data: IntegrationFormData) {
Copy link
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

when the integration is created using the existing identifier we just show the error toast but should additionally mark the field in the form and show the validation error there
Screenshot 2024-12-20 at 09 52 20

Copy link
Contributor Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Couldn't find a quick one for this, I don't think it's adds a lot value for the amount of work. Will skip it for now

Base automatically changed from feat-new-integrations-page-table to next December 20, 2024 11:40
@scopsy
Copy link
Contributor Author

scopsy commented Dec 20, 2024

@LetItRock merging to keep testing this out on staging with some legacy data, and cases under a FF

@scopsy scopsy merged commit e94f82c into next Dec 20, 2024
35 checks passed
@scopsy scopsy deleted the feat-integrations-page branch December 20, 2024 13:44
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants